-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Xch/test new nbgl #25
Conversation
Wiz Scan Summary
|
reject_confirmation); | ||
nbgl_useCaseReviewStreamingStart(TYPE_OPERATION, | ||
&C_stax_app_eos_64px, | ||
"Review operation", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we use TYPE_OPERATION
can't both string be defined in the SDK?
Maybe default behaviour could be to have them set to NULL, and if the developer want to customize he could still set it ?
pair.value = review_title; | ||
return &pair; | ||
} else { | ||
return get_single_action_review_pair(index -1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
weird spacing
|
||
void ui_display_single_action_sign_flow(void) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking at the code, I find it not so easy to follow functions to understand what the final screens will look like. Maybe we should try to find a descriptive way to comment the screens, such as what used to be done on bagl ? 🤔
pairList.startIndex = 0; | ||
|
||
nbgl_useCaseStaticReview(&pairList, &infoLongPress, "Reject transaction", review_choice); | ||
} | ||
if (txProcessingCtx.currentActionNumer == 1) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
numer 🔢
199f340
to
102fdca
Compare
Closing, because integrated, with latest NBGL fetures/API in #28 |
Demo of LedgerHQ/ledger-secure-sdk#576